Skip to content

Fix mark as paid feature in admin panel#1402

Open
OlegPhenomenon wants to merge 5 commits intomasterfrom
1329-mark-as-paid-doesnt-work
Open

Fix mark as paid feature in admin panel#1402
OlegPhenomenon wants to merge 5 commits intomasterfrom
1329-mark-as-paid-doesnt-work

Conversation

@OlegPhenomenon
Copy link
Contributor

@OlegPhenomenon OlegPhenomenon commented Jul 14, 2025

Close #1329

Refactor admin invoice show page and routes for better UX and Turbo compatibility

  • Refactored :

    • Simplified action buttons layout and updated button classes for consistency with the rest of the UI.
    • Replaced the info block grid with a clean two-column table for invoice details.
    • Unified table styles for invoice items and payment orders, using consistent custom classes.
    • Improved markup structure for clarity and maintainability.
  • Updated :

    • Added and under the scope for admin.
    • Removed legacy member route.
    • Ensured RESTful and Turbo-friendly routing for invoice actions.
  • Cleaned up controller logic (see diff) to match new routes and UI flow.

These changes improve the maintainability, consistency, and Turbo/Hotwire compatibility of the admin invoice management interface.

…ompatibility

- Refactored :
  - Simplified action buttons layout and updated button classes for consistency with the rest of the UI.
  - Replaced the info block grid with a clean two-column table for invoice details.
  - Unified table styles for invoice items and payment orders, using consistent custom classes.
  - Improved markup structure for clarity and maintainability.

- Updated :
  - Added  and  under the  scope for admin.
  - Removed legacy  member route.
  - Ensured RESTful and Turbo-friendly routing for invoice actions.

- Cleaned up controller logic (see diff) to match new routes and UI flow.

These changes improve the maintainability, consistency, and Turbo/Hotwire compatibility of the admin invoice management interface.
@OlegPhenomenon OlegPhenomenon force-pushed the 1329-mark-as-paid-doesnt-work branch from f30c9a7 to 64a4a6d Compare July 14, 2025 13:04
@OlegPhenomenon OlegPhenomenon force-pushed the 1329-mark-as-paid-doesnt-work branch from 131b762 to f246580 Compare July 14, 2025 13:22
@vohmar
Copy link
Contributor

vohmar commented Sep 26, 2025

admin can now change the invoice status in auction admin panel. But the status update does not get synchronized to billing

@vohmar vohmar assigned OlegPhenomenon and unassigned vohmar Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"Mark as paid" does not work

2 participants